-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update s2e-core pre v8 #91
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@200km こちらも、simulation実行時にerrorが出ていたので、local_environment.iniを修正してpushしました。また、HILS ON時にpower_controllser.cppでerrorが出ていたので、こちらも合わせて修正しています。SILS/HILSともにsimulationが回るのでapproveします。
実行時のエラー修正ありがとうございます。 こちら、マージはしないで良いですか? |
どちらでも大丈夫です。ただ、preという名前がついてますし、マージできるならしてしまっていいかと思うのですが、いかがでしょうか? |
s2e-core v8 のリリース前に v8 作業中のモノを使いたい、ということであれば、使いたい時点の s2e-core の commit で Pre Release を打っておくのがよいと思います(Git tag, GitHub Releases として) |
こちらで本格アップデート始めたのでこれはCLOSE |
Issue
NA
詳細
s2e-coreの最新の更新を取り込みたいという要求があったので、s2e-core v8 major update前だが、pre v8としてs2e-aobcに取り込んで更新作業を始める。
検証結果
CI結果参照
補足
NA